Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HDDS-11192. Increase SPNEGO URL test coverage #6956

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

adoroszlai
Copy link
Contributor

@adoroszlai adoroszlai commented Jul 16, 2024

What changes were proposed in this pull request?

Update existing Robot test for SPNEGO-enabled URLs to cover more endpoints (more servlets and more servers).

https://issues.apache.org/jira/browse/HDDS-11192

How was this patch tested?

Run in CI:

Verify SPNEGO URLs without auth                                       | PASS |
------------------------------------------------------------------------------
Verify SPNEGO URLs with auth                                          | PASS |

https://github.com/adoroszlai/ozone/actions/runs/9961163803/job/27522849505#step:5:997

$ grep -o 'curl.*http_code.*http://[^ ]*' ozonesecure.xml | wc -l
54

$ grep -o 'curl.*http_code.*http://[^ ]*' ozonesecure.xml | head 
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://om:9874/conf'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://om:9874/jmx'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://om:9874/logLevel'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://om:9874/logs/'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://om:9874/prom'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://om:9874/stacks'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://recon:9888/conf'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://recon:9888/jmx'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://recon:9888/logLevel'
curl --negotiate -u : --silent -o /dev/null -w '%{http_code}' 'http://recon:9888/logs/'

@adoroszlai adoroszlai self-assigned this Jul 16, 2024
@adoroszlai adoroszlai merged commit 7062e17 into apache:master Jul 17, 2024
26 checks passed
@adoroszlai
Copy link
Contributor Author

Thanks @jojochuang for the review.

@adoroszlai adoroszlai deleted the HDDS-11192 branch July 17, 2024 15:17
errose28 added a commit to errose28/ozone that referenced this pull request Jul 30, 2024
…-delete

* HDDS-10239-container-reconciliation: (184 commits)
  HDDS-10373. Implement framework for capturing Merkle Tree Metrics. (apache#6864)
  HDDS-11188. Initial setup for new UI layout and enable users to switch to new UI (apache#6953)
  HDDS-11120. Rich rebalancing status info (apache#6911)
  HDDS-11187. Fix Event Handling in Recon OMDBUpdatesHandler to Prevent ClassCastException. (apache#6950)
  HDDS-11213. Bump commons-daemon to 1.4.0 (apache#6971)
  HDDS-11212. Bump commons-net to 3.11.1 (apache#6973)
  HDDS-11211. Bump assertj-core to 3.26.3 (apache#6972)
  HDDS-11210. Bump log4j2 to 2.23.1 (apache#6970)
  HDDS-11150. Recon Overview page crashes due to failed API Calls (apache#6944)
  HDDS-11183. Keys from DeletedTable and DeletedDirTable of AOS should be deleted on batch operation while creating a snapshot (apache#6946)
  HDDS-11198. Fix Typescript configs for Recon (apache#6961)
  HDDS-11180. Simplify HttpServer2#inferMimeType return statement (apache#6963)
  HDDS-11194. OM missing audit log for upgrade (apache#6958)
  HDDS-10389. Implement a search feature for users to locate open keys within the Open Keys Insights section. (apache#6231)
  HDDS-10561. Dashboard for delete key metrics (apache#6948)
  HDDS-11192. Increase SPNEGO URL test coverage (apache#6956)
  HDDS-11179. DBConfigFromFile#readFromFile result of toIOException not thrown (apache#6957)
  HDDS-11186. First container log missing from bundle (apache#6952)
  HDDS-10844. Clarify snapshot create error message. (apache#6955)
  HDDS-11166. Switch to Rocky Linux-based ozone-runner (apache#6942)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants